-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Some cleanup. #303
Some cleanup. #303
Conversation
Removed code used only for testing & debugging purposes.
Looking good. BTW, do you know if you had any 3rd-party cron job entries in crontab when MerlinAU was executed? |
I for sure have cron jobs. I have things like spdmerlin and Backupmon and diversion and unbound all installed and scheduled with cron jobs configured for stuff. I don't think any of them were being executed at the time. I was only looking at the fact that it supressed the eject USB message. |
If you have those cron job entries then there should be log messages about removing them. Something is odd here. Can you send me a screenshot of your current list of scheduled cron jobs? |
I was just getting ready for bed, give me like 2 minutes I'll login for you real quick! |
Screenshot of: cru l Copy of the MerlinAU log file for that flash:
Contents of the crontab file located here: /var/spool/cron/crontabs
|
Great, thanks. Please don't stay up any longer only for this. There's no urgency. This can wait until tomorrow. |
Hmmm. I'm expecting the 3006.102.x F/W to behave the same WRT cron jobs, crontab, & cru command.... |
Now you got me curious... Looking at the script on the router I can confirm that the function RemoveCronJobsFromAddOns is there. Let me try again just for fun, the reason is I noticed in my log file for the MerlinAU run, it was at 4:44 AM, but my MerlinAU script has a modified date of 8AM. I'm wondering if it's possible my primary router just had an old copy of the dev version, maybe I never copied it to my primary after testing it on my AXE11000, and only once I made the changes to up the minimum version did I update the script and get the RemoveCronJobsFromAddOns function. |
This looks more promising ;) I likely had "1.3.1" with only the changelog changes, before your PR. Then updated it once I made my changes for my PR again lol. |
Good catch about the timestamp!! LOL!! |
I've had my head out of the game for a month, distracted with other things; clearly it works fine and I was just running "1.3.1" but I just had an outdated version lol. Checking the timeline of the modified time was my only hint, nothing in the code seemed weird to me. |
YES!!! Those are the log messages I would expect based on your cron jobs list. So, the function is working as designed on the newer F/W version as well.
Thanks for staying up to run the script & verify the "real" latest code. Now we both can go to sleep more relaxed. Have a great night, bud. Sleep tight :>) |
Removed code used only for testing & debugging purposes.